docs: sync CLAUDE.md and README with current source#111
Merged
Conversation
README: - Add `dot logout` command section. - Document deploy flags `--private`, `--contracts`, `--no-build`, and `--no-contract-build` in the main flags list. Collapse the duplicated CI flags table into a tighter checklist. - Update headless-mode summary so `--private` and `--contracts` are listed as independently optional alongside `--modable`. CLAUDE.md: - Fix `withSpan` overload count (was "2-arg + 4-arg", actual is "3-arg + 4-arg"). Spell out the signatures inline. - Note which commands pass `watchdog` and `hardExit` to `runCliCommand`. src/utils/deploy/storage.ts: - Fix stale module-level docstring that claimed we force `jsMerkle: true`. The wrapper deliberately does NOT set that flag (Kubo binary path), matching the in-function comment and the CLAUDE.md invariant. No runtime behaviour changes; no changeset.
Contributor
|
Dev build ready — try this branch: |
Contributor
E2E Test Pass · ✅ PASSTag:
Sentry traces: view spans for this run |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
dot logout, surfaces deploy flags--private,--contracts,--no-build,--no-contract-buildin the main flags list, and collapses the duplicated CI flags table.withSpanoverload count typo (was "2-arg + 4-arg", actual signatures are(op, name, fn)and(op, name, attributes, fn)) and notes which commands passwatchdog/hardExittorunCliCommand.src/utils/deploy/storage.ts: corrects a stale module-level docstring that claimedjsMerkle: trueis forced. The wrapper deliberately does NOT set that flag (Kubo binary path), matching the in-function comment and the CLAUDE.md invariant.No runtime behaviour changes. No changeset, since nothing user-visible shipped beyond doc accuracy.
Test plan
pnpm format:checknpx tsc --noEmitpnpm test